Skip to content

fix: make configuration rolldown compatible - #717

Merged
susnux merged 1 commit into
mainfrom
fix/make-rolldown-compatible
Sep 29, 2025
Merged

fix: make configuration rolldown compatible#717
susnux merged 1 commit into
mainfrom
fix/make-rolldown-compatible

Conversation

@susnux

@susnux susnux commented Sep 28, 2025

Copy link
Copy Markdown
Contributor
  • PreRenderedAsset.name is deprecated in rollup and completely removed in rolldown in favor of PreRenderedAsset.names.
  • manualChunks is deprecated in rolldown instead we should use advancedChunks.

@susnux
susnux requested review from ShGKme and st3iny September 28, 2025 12:11
@susnux susnux added type: bug 🐛 Something isn't working 3. to review 3️⃣ Waiting for reviews labels Sep 28, 2025
@susnux

susnux commented Sep 28, 2025

Copy link
Copy Markdown
Contributor Author

/backport to stable1

- `PreRenderedAsset.name` is deprecated in rollup and completely removed
  in rolldown in favor of `PreRenderedAsset.names`.
- `manualChunks` is deprecated in rolldown instead we should use `advancedChunks`.

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
@susnux
susnux force-pushed the fix/make-rolldown-compatible branch from 2e63557 to fc0a308 Compare September 28, 2025 12:15

@st3iny st3iny left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I believe this is backwards-compatible?

Code looks good.

@susnux

susnux commented Sep 28, 2025

Copy link
Copy Markdown
Contributor Author

I believe this is backwards-compatible?

yes - its even deprecated in rollup so also good for current state :)

@susnux
susnux merged commit f7153a1 into main Sep 29, 2025
10 checks passed
@susnux
susnux deleted the fix/make-rolldown-compatible branch September 29, 2025 08:52
@backportbot

backportbot Bot commented Sep 29, 2025

Copy link
Copy Markdown

The backport to stable1 failed. Please do this backport manually.

# Switch to the target branch and update it
git checkout stable1
git pull origin stable1

# Create the new backport branch
git checkout -b backport/717/stable1

# Cherry pick the change from the commit sha1 of the change against the default branch
# This might cause conflicts, resolve them
git cherry-pick fc0a3087

# Push the cherry pick commit to the remote repository and open a pull request
git push origin backport/717/stable1

Error: Failed to clone repository: Failed to clone and cache repo: EACCES: permission denied, mkdir '/app/cache/nextcloud-libraries/nextcloud-vite-config'


Learn more about backports at https://docs.nextcloud.com/server/stable/go.php?to=developer-backports.

@susnux

susnux commented Sep 29, 2025

Copy link
Copy Markdown
Contributor Author

/backport to stable1

@backportbot

backportbot Bot commented Sep 29, 2025

Copy link
Copy Markdown

The backport to stable1 failed. Please do this backport manually.

# Switch to the target branch and update it
git checkout stable1
git pull origin stable1

# Create the new backport branch
git checkout -b backport/717/stable1

# Cherry pick the change from the commit sha1 of the change against the default branch
# This might cause conflicts, resolve them
git cherry-pick fc0a3087

# Push the cherry pick commit to the remote repository and open a pull request
git push origin backport/717/stable1

Error: Failed to clone repository: Failed to clone and cache repo: EACCES: permission denied, mkdir '/app/cache/nextcloud-libraries/nextcloud-vite-config'


Learn more about backports at https://docs.nextcloud.com/server/stable/go.php?to=developer-backports.

@susnux

susnux commented Sep 29, 2025

Copy link
Copy Markdown
Contributor Author

/backport to stable1

@skjnldsv

Copy link
Copy Markdown
Contributor

/backport to stable1

@backportbot

backportbot Bot commented Sep 29, 2025

Copy link
Copy Markdown

The backport to stable1 failed. Please do this backport manually.

# Switch to the target branch and update it
git checkout stable1
git pull origin stable1

# Create the new backport branch
git checkout -b backport/717/stable1

# Cherry pick the change from the commit sha1 of the change against the default branch
# This might cause conflicts, resolve them
git cherry-pick fc0a3087

# Push the cherry pick commit to the remote repository and open a pull request
git push origin backport/717/stable1

Error: Failed to create pull request: Validation Failed: {"resource":"PullRequest","code":"custom","message":"A pull request already exists for nextcloud-libraries:backport/717/stable1."} - https://docs.github.com/rest/pulls/pulls#create-a-pull-request


Learn more about backports at https://docs.nextcloud.com/server/stable/go.php?to=developer-backports.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review 3️⃣ Waiting for reviews type: bug 🐛 Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants